-
Notifications
You must be signed in to change notification settings - Fork 970
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
.NET 9 Update to fix armv7l breakage #3573
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Satadru Pramanik <satadru@gmail.com>
This updates .Net to 9.0.100-rc.2.24474.11 . See actions#3505 dotnet/runtime#101444 dotnet/runtime#96460 dotnet/runtime#102410 (Looks like this has not been backported to .Net 8.x...) Signed-off-by: Satadru Pramanik <satadru@gmail.com>
Signed-off-by: Satadru Pramanik <satadru@gmail.com>
Signed-off-by: Satadru Pramanik <satadru@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Updating .NET to version 9 is necessary to keep supporting armv7 as .NET won't fix the incompatibilty in version 8.
I assume we will lose the support for ubuntu 20.04 or debian 11 for arm32? |
users of these versions might have to install .NET 9.0 from the Microsoft repository. I will check if thats reasonable |
https://github.com/dotnet/core/blob/main/release-notes/8.0/supported-os.md#linux-compatibility Basically, the dotnet 8 runner should work on ubuntu 22.04 or below, just not ubuntu 24.04 or above is what my understand. |
That is also my understanding. The newer breakage is only with 32-bit runners. (We don't have this issue with our x86 32-bit runner because we run that inside a x86_64 based container...) |
I eventually also came to that understanding. We are propably able to stay on jammy so I am trying this out right now |
Fixes #3505